Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat: powershell target #515

Merged
merged 21 commits into from
Mar 11, 2024
Merged

Conversation

vaind
Copy link
Contributor

@vaind vaind commented Feb 26, 2024

This adds powershell publishing support

@vaind vaind changed the title feat: powershell publishing feat: powershell target Feb 27, 2024
Copy link
Member

@asottile-sentry asottile-sentry left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

can you write some tests for your new feature

src/utils/version.ts Outdated Show resolved Hide resolved
src/utils/version.ts Outdated Show resolved Hide resolved
Dockerfile Outdated Show resolved Hide resolved
@vaind
Copy link
Contributor Author

vaind commented Feb 27, 2024

can you write some tests for your new feature

The PR is still a draft (a.k.a not ready for review) and the unrelated stuff will be split out + I hope to add tests if I can figure it out. Currently I'm testing it manually and trying to fix unrelated issues with zip file download

@vaind vaind marked this pull request as ready for review February 29, 2024 18:53
@bruno-garcia
Copy link
Member

Thanks @vaind for raising this!

@bruno-garcia
Copy link
Member

@asottile-sentry please let me know if we need any other change to get this through

Copy link
Member

@asottile-sentry asottile-sentry left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

apologies I was out sick the last week and change (still am) but going to approve to unblock

@vaind
Copy link
Contributor Author

vaind commented Mar 11, 2024 via email

@bruno-garcia bruno-garcia merged commit 3de62e6 into getsentry:master Mar 11, 2024
6 checks passed
@bruno-garcia
Copy link
Member

Thanks @vaind and @asottile-sentry !

@bruno-garcia
Copy link
Member

@asottile-sentry hope u get well soon 🙏

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants